Store headstage sensor data as separate TimeSeries objects with appropriate units and scaling #142
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR implements separate storage of headstage sensor data (accelerometer, gyroscope, magnetometer) as individual TimeSeries objects in the acquisition module with proper physical units and scaling factors, addressing the issue where all analog data was previously combined into a single TimeSeries with generic units.
Key Changes
Enhanced Analog Data Processing:
Headstage_AccelX/Y/Z→ Accelerometer data scaled by 0.000061 with unitgHeadstage_GyroX/Y/Z→ Gyroscope data scaled by 0.061 with unitd/sHeadstage_MagX/Y/Z→ Magnetometer data with unitunspecifiedECU_Ain*→ Analog input channels with configurable unitsMetadata Schema Extensions:
sensor_unitsfield for custom unit specifications per sensor typebehavioral_eventsitems to support individualunitandcommentsfieldsOutput Structure:
nwbfile.acquisitionwith descriptive names and proper unitsseparate_sensor_data=Falseparameter preserves original combined TimeSeries in processing moduleExample Usage:
Benefits:
Testing:
Fixes #19.
💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.